Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: define interfaces for file and project generation #1568

Merged
merged 10 commits into from
Sep 1, 2023

Conversation

mdelapenya
Copy link
Member

@mdelapenya mdelapenya commented Sep 1, 2023

What does this PR do?

This PRs renames the entities from Example to TestcontainersModule in order to be more intentional.

At the same time, creates two interfaces: FileGenerator and ProjectGenerator. The first one will generate files for a module when added, and the second one will inspect the state of the file system in order to generate the project files (CI workflow and VSCode workspace).

Why is it important?

Better abstractions

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner September 1, 2023 10:07
@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Sep 1, 2023
@mdelapenya mdelapenya self-assigned this Sep 1, 2023
@netlify
Copy link

netlify bot commented Sep 1, 2023

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 2d03cfe
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/64f1bee4f6cede00084c5825
😎 Deploy Preview https://deploy-preview-1568--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

modulegen/internal/context/context.go Show resolved Hide resolved
modulegen/internal/main.go Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Sep 1, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.6% 0.6% Duplication

@mdelapenya mdelapenya merged commit 69a5d38 into testcontainers:main Sep 1, 2023
93 checks passed
@mdelapenya mdelapenya deleted the add-examples-to-modules branch September 1, 2023 11:31
mdelapenya added a commit to mdelapenya/testcontainers-go that referenced this pull request Sep 1, 2023
* main:
  chore: define interfaces for file and project generation (testcontainers#1568)
  chore(deps): bump mkdocs-material from 8.2.7 to 9.2.6 (testcontainers#1567)
  chore(deps): bump the all group in /modules/neo4j with 1 update (testcontainers#1562)
  chore(deps): bump the all group in /modules/clickhouse with 1 update (testcontainers#1563)
  chore(deps): bump the all group in /modules/localstack with 1 update (testcontainers#1564)
  Update actions/github-script to 6.4.1 (testcontainers#1561)
  chore: Update Pipenv to use Python 3.8 (testcontainers#1558)
  feat: convert NATS example into a module (testcontainers#1559)
  chore(deps): bump the all group in /modulegen with 1 update (testcontainers#1554)
  modulegen: use cobra instead of native golang flag (testcontainers#1550)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes that do not impact the existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants